-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New --ouputDiffLines
cli flag
#1446
Conversation
✅ Deploy Preview for vitest-dev ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site settings. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍🏻 But left some notes for docs
docs/config/index.md
Outdated
Limit number of output diff lines up to `15`. You may wish to tune this, to see output | ||
for larger objects/arrays. | ||
|
||
To provide object via CLI command, use the following syntax: `--outputDiffLines=100`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what object? 😄
I think you've taken it from outputFile
command, where it's possible to provide object via CLI, but here it' just a number. I would remove this line and on :248
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I meant, I could see whole objects/arrays in the Diff output, not that objects can be supplied to the cli flag. I will make it clear. Thanks!
Use this flag in order to control how many lines are printed, on each diff.
Using this flag enables me to see whole objects/arrays I am testing, allowing me to copy/paste them.
Also adds documentation for previous feature
--outputTruncateLength